-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
treewide: migrate packages to by-name hierarchy #254417
treewide: migrate packages to by-name hierarchy #254417
Conversation
I think it was discussed at NixCon that we should move to the new schema using automation (where possible) to avoid the overhead of many PRs. You also did some cleanup so in this case this is probably fine. |
This is an opportunity to review my own code and not lose some tracks. I have found some packages that should be removed and others needing more attention, like TLOE. Also there is no problem in some manual intervention. |
@ofborg build pkgsLLVM.mksh mksh |
Yeah so I will work on updating #211832 next, which is the automated mass migration. I wouldn't want people to have to do this manually to save on both reviewer and contributor time. I think it's fine if you happen to clean up the code or update the versions anyways though. Since you did the work here already I guess it's fine, but others should definitely not be encouraged to do this manually until the automated migration is done (it doesn't work automatically for all packages) 🙂 |
I opened #254632 to have this written down |
OK, I will stop this thread for now and concentrate in the version bumps. |
@ofborg build xorriso dosbox-staging |
Are you intending on having this reviewed and merged after all? Or are you using this as a testing ground before splitting it into smaller cleanup PRs? If so this should probably be turned into a draft. |
Both |
- multiple outputs - add meta.mainProgram - migrate to by-name
- add meta.mainProgram - migrate to by-name
- rec-less
- finalAttrs - no nested with
- factor out tests
Description of changes
The ones maintained by @AndersonTorres
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)